Skip to content

fix: align deprecation warnings with transition policy - #30

Open
BaruqueRodrigues wants to merge 2 commits into
codex/pr2-legacy-compatfrom
codex/pr4-deprecation-policy
Open

BaruqueRodrigues wants to merge 2 commits into
codex/pr2-legacy-compatfrom
codex/pr4-deprecation-policy

Conversation

@BaruqueRodrigues

Copy link
Copy Markdown

Problema

A política aprovada define o início dos avisos formais de depreciação na versão 1.3.0, mas os wrappers já avisavam na 1.2.0.

Alterações

  • Emite avisos lifecycle somente quando a versão carregada do pacote atinge 1.3.0.
  • Alinha o guia de migração e o NEWS à política aprovada.
  • Atualiza testes para garantir que a versão 1.2.0 não antecipe os avisos.

Validação

devtools::test(filter = "api", stop_on_failure = TRUE) — 56 verificações passaram.

PR empilhada sobre a PR #29.

@BaruqueRodrigues
BaruqueRodrigues force-pushed the codex/pr4-deprecation-policy branch from f2c581b to 8c059d5 Compare October 20, 2026 13:58
@BaruqueRodrigues
BaruqueRodrigues force-pushed the codex/pr3-setcriteria-compat branch from 8a27117 to e76044d Compare October 20, 2026 14:06
@BaruqueRodrigues
BaruqueRodrigues force-pushed the codex/pr4-deprecation-policy branch from 8c059d5 to 0fd9c0d Compare October 20, 2026 14:06
Base automatically changed from codex/pr3-setcriteria-compat to codex/pr2-legacy-compat October 27, 2026 17:00
@esloch

esloch commented Oct 27, 2026

Copy link
Copy Markdown
Collaborator

@BaruqueRodrigues,

Tenho apenas dois pontos para ajustarmos antes do merge.

  1. Os testes verificam que a versão 1.2.0 não emite avisos, mas ainda precisamos validar que eles serão emitidos a partir da 1.3.0. Sugiro separar a verificação da versão para permitir testar os dois cenários.

  2. No [api-migration.md](https://github.com/AlertaDengue/AlertTools/blob/0fd9c0da8f3d830d115f7034102f74038f63aefb/docs/refactoring/api-migration.md), ainda consta que a política de suporte está proposta. Entretanto, o [transition-policy.dcf](https://github.com/AlertaDengue/AlertTools/blob/0fd9c0da8f3d830d115f7034102f74038f63aefb/docs/refactoring/transition-policy.dcf) já registra sua aprovação. Precisamos alinhar essa informação.

Também seria bom atualizar a branch com as alterações do PR #29, que já foi integrado, para manter o diff restrito ao escopo desta correção.

@esloch esloch left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Temos alguns ajustes nos testes e na documentação.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants